Skip to content

docs: update CONTRIBUTING.md with merge strategy preference and AI manifesto link#2756

Merged
endigma merged 3 commits intomainfrom
endigma/contributing-merge-strategy
Apr 10, 2026
Merged

docs: update CONTRIBUTING.md with merge strategy preference and AI manifesto link#2756
endigma merged 3 commits intomainfrom
endigma/contributing-merge-strategy

Conversation

@endigma
Copy link
Copy Markdown
Member

@endigma endigma commented Apr 10, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Documentation
    • Added two contribution guidelines: align contributions with the Open Source AI Manifesto and prefer merging (to preserve review history) over rebasing after review requests.
  • Chores
    • Expanded PR trigger to run the build/test workflow when repository metadata or documentation files are modified.

Add a note to CONTRIBUTING.md preferring merge strategy over rebase after review has been requested. Add a link to the Open Source AI Manifesto, consistent with README.md and the PR template.

Checklist

  • I have discussed my proposed changes in an issue and have received approval to proceed.
  • I have followed the coding standards of the project.
  • Tests or benchmarks have been added or updated.
  • Documentation has been updated on https://github.com/wundergraph/docs-website.
  • I have read the Contributors Guide.

Open Source AI Manifesto

This project follows the principles of the Open Source AI Manifesto. Please ensure your contribution aligns with its principles.

@endigma
docs: update CONTRIBUTING.md with merge strategy preference and AI ma…
68dd4a4 
…nifesto link

Add note preferring merge over rebase after review is requested.
Add link to the Open Source AI Manifesto.
@endigma endigma requested review from a team as code owners April 10, 2026 14:20
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 11ef951f-ce6a-4824-8cae-7adb2be62a02

📥 Commits

Reviewing files that changed from the base of the PR and between 68dd4a4 and 8fb9c79.

📒 Files selected for processing (1)
  • .github/workflows/dummy-ci.yaml
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/dummy-ci.yaml

Walkthrough

Updated CONTRIBUTING.md with two new contribution guidelines and expanded the pull_request path filters in .github/workflows/dummy-ci.yaml to include several repository root and metadata files so PRs touching those files trigger the build_test job.

Changes

Cohort / File(s) Summary
Contribution guidelines
CONTRIBUTING.md		 Added two rules: contributions must align with the Open Source AI Manifesto principles, and PR branch updates should prefer merging (e.g., git merge main) over rebasing to preserve review context and avoid force-pushes.
CI workflow trigger
.github/workflows/dummy-ci.yaml		 Expanded on.pull_request.paths to include repository root and metadata files (CLAUDE.md, CONTRIBUTING.md, README.md, SECURITY.md, LICENSE, .github/PULL_REQUEST_TEMPLATE.md) so PRs modifying those files run the existing build_test job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes: updating CONTRIBUTING.md with merge strategy preference and AI manifesto link, which aligns with the actual modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Comment @coderabbitai help to get the list of available commands and usage tips.

@endigma endigma merged commit bb537ba into main Apr 10, 2026
9 checks passed
@endigma endigma deleted the endigma/contributing-merge-strategy branch April 10, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ysmolski ysmolski ysmolski approved these changes

@comatory comatory comatory approved these changes

@thisisnithin thisisnithin Awaiting requested review from thisisnithin thisisnithin is a code owner automatically assigned from wundergraph/cosmo

@Aenimus Aenimus Awaiting requested review from Aenimus Aenimus is a code owner automatically assigned from wundergraph/cosmo

@StarpTech StarpTech Awaiting requested review from StarpTech StarpTech is a code owner automatically assigned from wundergraph/router

@SkArchon SkArchon Awaiting requested review from SkArchon SkArchon is a code owner automatically assigned from wundergraph/router

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@endigma @ysmolski @comatory